-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/random music #13
Conversation
코드를 리포맷팅한 커밋이 실수로 포함되었네요 ㅠㅠ |
색상은 Color Hunt 를 이용했습니다. :] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다. 크게 문제 없는 것 같아요 :) !
|
저도 디자인 작업 시 해당 색상 위주로 사용해야겠네요:) 감사합니다. |
This reverts commit d08979f.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다:] 고생하셨네요!
LiarGame/Sources/ViewController/RandomMusicQuiz/DashedLineBorderdLabel.swift
Outdated
Show resolved
Hide resolved
LiarGame/Sources/ViewController/RandomMusicQuiz/RandomMusicQuizViewController.swift
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코드 읽느라 고생이 많으시네요....
몇몇 부분 재 피드백 부탁드려요 :]
해소된 부분은 Resolve conversation
!
LiarGame/Sources/ViewController/RandomMusicQuiz/RandomMusicQuizViewController.swift
Show resolved
Hide resolved
LiarGame/Sources/ViewController/RandomMusicQuiz/DashedLineBorderdLabel.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했어요~
- 재사용 않는 클래스 final 로 변경 - 모델 파일 분리 - 네이밍 수정
- 무한로딩되던 문제 해결 - ytPlayer 라이브러리에서 delegate 호출이 정상적으로 이루어지지 않아 발생 - ytPlayer 로의 요청중일 때 막는 방식으로 해결
비디오 재생과정은 아래와 같다. 1. 비디오 로드 // pending 2. 비디오 준비 // ready 3. 비디오 재생 명령 // .startVideo() 4-1. 비디오 버퍼링 // buffering 4-2. 비디오 재생 비디오 재생을 수행 시 버퍼링 과정이 포함되는데, 버퍼링 -> 재생까지의 시각이 매번 달라 YTPlayer 의 state 를 확인하여 재생되는 시점을 확인하고 재생되도록 하였음. - 다른 state (isReady) 와 같은 state 는 PlayerState 로 관리하도록 수정 - 재생 / 정지 동작을 명령할 state 네이밍 수정 fix 2jae6#8
시간초 재생 후 시작을 눌렀을 때 시간초 재생이 이루어지던 오류 수정
무한로딩 발생하던 문제
재생 시간이 매번 다르던 문제,시간초가 포함된 재생시 버퍼링으로 인해 재생 시간이 매번 달랐습니다. |
확인했어요 !!! 너무 수고 많으셨습니다 :) |
구현
#8 기능 구현
장르를 선택하는 장르 영역을 두었습니다.
아래 영상은 시연을 위해 유튜브 뷰를 보여지도록 했으나 원래는
hidden
상태로 동작합니다.오류
로딩이 시작되었을 때 간헐적으로 로딩이 끝나지 않는 문제영상 재생 시간이 일관적이지 않은 문제유튜브 비디오가 간헐적으로 로드되지 않는 문제흐름
최초 실행 시
재생
isPlaying
상태 변경false
셔플
answer
,isPlaying
상태 변경isLoading
true 로 변경ytPlayer.rx.isReady
일때에playerReady
액션 방출playerReady
시에isLoading
false 로 변경업데이트
answer
,isPlaying
상태 변경isLoading
true 로 변경ytPlayer.rx.isReady
일때에playerReady
액션 방출playerReady
시에isLoading
false 로 변경예시
ouput.mp4